-
-
Notifications
You must be signed in to change notification settings - Fork 282
[17.0][MIG] l10n_br_account #4339
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 17.0
Are you sure you want to change the base?
Conversation
|
Hi @renatonlima, |
e71ded0 to
c1bbed3
Compare
|
/ocabot migration l10n_br_account |
|
The migration issue (#2771) has not been updated to reference the current pull request because a previous pull request (#3757) is not closed. |
|
/ocabot migration l10n_br_account |
7312298 to
e55ddc5
Compare
| env["account.chart.template"].try_loading(company.chart_template, company) | ||
| todo_companies.append(company) | ||
| elif company in br_demo_companies: # fallback to generic_coa | ||
| env["account.chart.template"].try_loading("generic_coa", company) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rvalyi, se tentar carregar o generic coa antes da definição da moeda da erro na instalação do l10n_br_account:
2026-01-15 19:11:26,953 22 INFO devel odoo.modules.loading: Module l10n_br_account: loading demo
2026-01-15 19:11:26,954 22 INFO devel odoo.modules.loading: loading l10n_br_account/demo/res_users_demo.xml
2026-01-15 19:11:30,718 22 WARNING devel odoo.modules.loading: Transient module states were reset
2026-01-15 19:11:30,759 22 ERROR devel odoo.modules.registry: Failed to load registry
Traceback (most recent call last):
File "/opt/odoo/custom/src/odoo/odoo/modules/registry.py", line 110, in new
odoo.modules.load_modules(registry, force_demo, status, update_module)
File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 485, in load_modules
processed_modules += load_marked_modules(env, graph,
File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 366, in load_marked_modules
loaded, processed = load_module_graph(
File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 247, in load_module_graph
getattr(py_module, post_init)(env)
File "/opt/odoo/auto/addons/l10n_br_account/hooks.py", line 21, in post_init_hook
company.currency_id = env.ref("base.BRL")
File "/opt/odoo/custom/src/odoo/odoo/fields.py", line 1334, in __set__
records.write({self.name: write_value})
File "/opt/odoo/auto/addons/account/models/company.py", line 412, in write
raise UserError(_('You cannot change the currency of the company since some journal items already exist'))
odoo.exceptions.UserError: You cannot change the currency of the company since some journal items already exist
2026-01-15 19:11:30,761 22 CRITICAL devel odoo.service.server: Failed to initialize database `devel`.
Traceback (most recent call last):
File "/opt/odoo/custom/src/odoo/odoo/service/server.py", line 1374, in preload_registries
registry = Registry.new(dbname, update_module=update_module)
File "<decorator-gen-16>", line 2, in new
File "/opt/odoo/custom/src/odoo/odoo/tools/func.py", line 87, in locked
return func(inst, *args, **kwargs)
File "/opt/odoo/custom/src/odoo/odoo/modules/registry.py", line 110, in new
odoo.modules.load_modules(registry, force_demo, status, update_module)
File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 485, in load_modules
processed_modules += load_marked_modules(env, graph,
File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 366, in load_marked_modules
loaded, processed = load_module_graph(
File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 247, in load_module_graph
getattr(py_module, post_init)(env)
File "/opt/odoo/auto/addons/l10n_br_account/hooks.py", line 21, in post_init_hook
company.currency_id = env.ref("base.BRL")
File "/opt/odoo/custom/src/odoo/odoo/fields.py", line 1334, in __set__
records.write({self.name: write_value})
File "/opt/odoo/auto/addons/account/models/company.py", line 412, in write
raise UserError(_('You cannot change the currency of the company since some journal items already exist'))
odoo.exceptions.UserError: You cannot change the currency of the company since some journal items already exist
2026-01-15 19:11:30,762 22 INFO devel odoo.service.server: Stopping gracefully
Traceback (most recent call last):
File "/usr/local/bin/addons", line 153, in <module>
check_call(command)
File "/usr/local/lib/python3.10/subprocess.py", line 369, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['odoo', '--stop-after-init', '--init', 'l10n_br_base,l10n_br_account']' returned non-zero exit status 255.
Consegui resolver apenas trocando a orderm aqui, veja:
elif company in br_demo_companies: # fallback to generic_coa
company.currency_id = env.ref("base.BRL") # define moeda primeiro <--
env["account.chart.template"].try_loading("generic_coa", company) # essa linha gera account.move.lines <--
todo_companies.append(company)
EDIT:
Trocar a ordem gera um outro problema, a moeda não está sendo definida e permanece USD

e55ddc5 to
74c08a7
Compare
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-brazil-16.0/l10n-brazil-16.0-l10n_br_account Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-16-0/l10n-brazil-16-0-l10n_br_account/
Currently translated at 15.1% (118 of 779 strings) Translation: l10n-brazil-16.0/l10n-brazil-16.0-l10n_br_account Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-16-0/l10n-brazil-16-0-l10n_br_account/pt_BR/
…before popup as it changes in v17
74c08a7 to
6b42a8f
Compare
|
@DiegoParadeda fiz um rebase agora que algumas mudanças importantes jà entraram na 17.0. Se vc quiser ajudar que tal vc tentar fazer o PR da migração do modulo l10n_br_coa_generic para a 17.0 primeiro? So que se vc for fazer, por favor faz um cherry pick das evoluções do modulo que eu tinha começado a fazer na 16.0 aqui #3804 (pode ser que precisa emendar uma coisa ou outra pois já faz um certo tempo que eu tinha feito esse PR). |
Work In Progress
depende de #4018
O l10n_br_account instala e funciona minimamente qdo instala, so que no momento precisa salvar o account.move antes de adicionar linhas account.move.line.
2 meses atras isso funcionava mas algo quebrou no rebase que eu dei ou com cherry-picks que eu fiz, vou analisar isso ainda..
A prioridade tem que ser de ter 100% dos testes passando (resolvando a regressão com a edição das linhas, teria uns 70% passando, é o que tinha 2 meses atrás). Falta tb alguns commits recentes para fazer cherry pick ainda, da reforma em especial, mas talvez é bom ver isso depois de corrigir a regressão da edição.
Aceito PRs de ajuda no repo da Akretion mas eu peço ir com calma pois vai ter que ir dando rebase e squash de muita coisa ainda, em especial os commits com mensagem WIP que eu fiz. Novamente sou eu que fiz uns 95% dos ports neste modulo esses 15 últimos anos e só é possível fazer com histórico de migração limpo, senao os conflitos de merge explodem.
Traduzindo: ajuda de iniciante não é bem ajuda, tem muita coisa que iniciante pode fazer, até em outro repo para provar que na real é foda para caralh# antes de conseguir ajudar de verdade com esse tipo de módulo muito complexo...
Com isso pode facilmente levar mais 1 mes para mesclar esse PR (espero que não porem, mas sendo realista). Então para migrar para 18.0 e mesclar vai levar mais tempo ainda... Como sempre, podem tb contratar horas se quiserem mais velocidade apesar que não tou tentendo mendigar isso aqui, pois ja estamos bem atarefados.